Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addtelestompfix #1343

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

solarisiko232
Copy link

please check if this fix is applicable! I am using it right now but not 100% sure if it is completely crashless

@noah-
Copy link
Collaborator

noah- commented Mar 16, 2019

hi, I think easiest method would be to change the sortFunc such that telestomp candidates are sorted to the end

as for moving the sort outside the loop, I think it was inside the loop because it was also distanced based and monsters move around

@solarisiko232
Copy link
Author

okay, ty! I will work on it a bit more and check in again

@ryancrunchi
Copy link
Contributor

Won't this fix break with baal wave 2 with hammer where bosses spawn skeletons ? Won't it prioritize skeletons instead of telestomping immunes spawners first ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants